Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Turn page while playing #1383

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

RaHoni
Copy link
Contributor

@RaHoni RaHoni commented Nov 17, 2024

This is a fix for #1365. This adds an additional settings that disables the check for playing when enabled.
The default for this setting is off.

But since while testing i noticed that the problem for #1374 was also showing up on the Virtual Pixel 7a. The simple fix was removing the half-screen option. @advplyr this does change the Settings to be full-screen. Is this okay or is there something else I should do?

This allows to allow the navigation with the Volume buttons even when
audiobooks are playing.
@RaHoni RaHoni force-pushed the turn_page_while_playing branch from 34e9225 to 12a3969 Compare November 17, 2024 22:47
@RaHoni
Copy link
Contributor Author

RaHoni commented Nov 21, 2024

@fawazral would you want to test this PR?

@fawazral
Copy link

@RaHoni I just tested it out and it works as intended! Thanks so much for doing this!

@advplyr
Copy link
Owner

advplyr commented Dec 12, 2024

I didn't catch this before but I think we can use the same "On" and "Off" translation for the settings. If it turns out there are some languages that need them separated for whatever reason we can do that, but if we can re-use translation strings we should.

I updated the label for the new ereader setting to be more clear so you'll have to translate that again in German

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants